Skip to content

Bump Go to 1.26.1 and golangci-lint to 2.11.3#499

Merged
ntnn merged 6 commits into
kbind-dev:mainfrom
cnvergence:bump-go-ver
Mar 30, 2026
Merged

Bump Go to 1.26.1 and golangci-lint to 2.11.3#499
ntnn merged 6 commits into
kbind-dev:mainfrom
cnvergence:bump-go-ver

Conversation

@cnvergence

@cnvergence cnvergence commented Mar 20, 2026

Copy link
Copy Markdown
Member

Summary

This PR will update Go version to 1.26.1, fixing recent CVEs.
We also need to bump golangci-lint, as build version was 1.24 in the previous golangci-lint version, this required fixing linter issues across our codebase.

What Type of PR Is This?

Related Issue(s)

Fixes #

Release Notes

NONE

@cnvergence cnvergence requested a review from a team as a code owner March 20, 2026 19:02
Comment thread go.mod Outdated
// go-build-version 1.25.4
go 1.24.0
// go-build-version 1.25.8
go 1.25.8

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The minimum should always be the .0, or?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should be equal, this is used for go version checking script to ensure all versions are consistent

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The go-build-version should be the version we build with, the go version itself should just be the major version, we do it in kcp the same way (unless there's a security issue that requires us to use a later go version)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see, then lets do it the same way as in the kcp

@ntnn ntnn added this to tbd Mar 20, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in tbd Mar 20, 2026
@ntnn ntnn moved this from Backlog to Reviewing in tbd Mar 28, 2026
@cnvergence cnvergence changed the title Bump Go to 1.25.8 Bump Go to 1.26.1 and golangci-lint to 2.11.3 Mar 30, 2026
Comment thread .golangci.yaml
Comment thread pkg/konnector/controllers/cluster/serviceexport/serviceexport_reconcile.go Outdated
Comment thread test/e2e/framework/dex.go Outdated
ntnn
ntnn previously approved these changes Mar 30, 2026

@ntnn ntnn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@ntnn ntnn enabled auto-merge March 30, 2026 14:22
@ntnn ntnn merged commit be1021f into kbind-dev:main Mar 30, 2026
12 of 13 checks passed
@github-project-automation github-project-automation Bot moved this from Reviewing to Done in tbd Mar 30, 2026
@cnvergence cnvergence deleted the bump-go-ver branch March 30, 2026 15:24
@ntnn ntnn moved this from Done to Reviewing in tbd Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants